Skip to content

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented Jun 27, 2025

This PR implements comprehensive documentation and UI support for remote hubs (cross-tenant) functionality in FinOps hubs.

Problem

Remote hubs support was added to the backend infrastructure but was never fully documented or accessible through the UI deployment form. Users had no guidance on how to configure cross-tenant scenarios where satellite hubs send data to a central primary hub.

Solution

1. UI Form Enhancement

  • Added Remote hub configuration section to the Advanced step in createUiDefinition.json
  • Added remoteHubStorageUri field with Data Lake storage URI validation
  • Added remoteHubStorageKey field as secure password input with storage key validation
  • Properly configured outputs to pass parameters to the deployment template

2. Comprehensive Documentation

  • Added new section "Optional: Configure remote hubs" to the main deployment guide
  • Documented when to use remote hubs (multi-tenant scenarios)
  • Explained architecture with primary and satellite hub concepts
  • Provided step-by-step configuration for both Azure portal and PowerShell
  • Included security considerations and best practices

3. PowerShell Documentation Updates

  • Updated Deploy-FinOpsHub command syntax to include remote hub parameters
  • Enhanced parameters table with detailed descriptions and examples
  • Added complete deployment example for remote hub scenarios

4. Enhanced Template Documentation

  • Improved parameter descriptions for remoteHubStorageUri and remoteHubStorageKey
  • Added clear instructions for obtaining values from the primary hub
  • Documented version requirements (0.4+) and security implications

Key Features

  • Cross-tenant support: Enable cost data aggregation across multiple Azure tenants
  • Secure configuration: Proper handling of storage keys as secure parameters
  • Complete guidance: End-to-end instructions for both portal and PowerShell deployment
  • Architecture clarity: Clear explanation of primary vs satellite hub roles

Configuration Parameters

  • remoteHubStorageUri: Data Lake storage endpoint from primary hub (e.g., https://primaryhub.dfs.core.windows.net/)
  • remoteHubStorageKey: Storage account access key from primary hub

Usage Example

Deploy-FinOpsHub `
    -Name MyRemoteHub `
    -ResourceGroup MyRemoteHubResourceGroup `
    -Location westus `
    -RemoteHubStorageUri "https://centralhub.dfs.core.windows.net/" `
    -RemoteHubStorageKey "storage-key-here"

The backend implementation already existed - this PR completes the user-facing documentation and UI integration to make remote hubs fully accessible and properly documented.

Fixes #1093.


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

@Copilot Copilot AI changed the title [WIP] [Hubs] Document remote hubs support [Hubs] Document remote hubs support and add UI configuration Jun 27, 2025
Copilot finished work on behalf of flanakin June 27, 2025 07:26
@flanakin flanakin added this to the 2025-06 - June milestone Jun 27, 2025
@microsoft-github-policy-service microsoft-github-policy-service bot added Needs: Attention 👋 Issue or PR needs to be reviewed by the author or it will be closed due to no activity and removed Needs: Review 👀 PR that is ready to be reviewed labels Jun 27, 2025
@microsoft-github-policy-service microsoft-github-policy-service bot added Needs: Review 👀 PR that is ready to be reviewed and removed Needs: Attention 👋 Issue or PR needs to be reviewed by the author or it will be closed due to no activity labels Jun 27, 2025
Copilot finished work on behalf of flanakin June 27, 2025 15:42
@flanakin flanakin removed this from the 2025-06 - June milestone Jul 6, 2025
@flanakin flanakin added this to the 2025-07 - July milestone Jul 6, 2025
@MSBrett MSBrett marked this pull request as ready for review October 11, 2025 21:11
@MSBrett MSBrett merged commit 7fa7896 into dev Oct 11, 2025
5 checks passed
@MSBrett MSBrett deleted the copilot/fix-1093 branch October 11, 2025 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs: Review 👀 PR that is ready to be reviewed Skill: DevOps GitHub setup and automation Skill: Documentation Documentation updates Tool: FinOps guide Implementing FinOps guide

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Hubs] Document remote hubs support

6 participants